Skip to content

bump k8s-interface to v0.0.205 and update renamed constants#752

Closed
idohuber wants to merge 1 commit intomainfrom
fix/bump-k8s-interface-renamed-constants
Closed

bump k8s-interface to v0.0.205 and update renamed constants#752
idohuber wants to merge 1 commit intomainfrom
fix/bump-k8s-interface-renamed-constants

Conversation

@idohuber
Copy link

@idohuber idohuber commented Mar 17, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Chores
    • Updated dependencies to the latest versions.
  • Bug Fixes
    • Improved accuracy of related-resource labels so displayed relationships are more reliable.

@coderabbitai
Copy link

coderabbitai bot commented Mar 17, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f08c2047-38f4-4354-96a5-a8c8f03f0330

📥 Commits

Reviewing files that changed from the base of the PR and between f00b41a and bcc99e3.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (2)
  • go.mod
  • pkg/objectcache/shared_container_data.go
🚧 Files skipped from review as they are similar to previous changes (1)
  • go.mod
📝 Walkthrough

Walkthrough

Dependency version bumps in go.mod (multiple packages including kubescape and OpenTelemetry) and a change in GetLabels to use RelatedKindMetadataKey and RelatedNameMetadataKey with corresponding WLID helper getters.

Changes

Cohort / File(s) Summary
Dependency Management
go.mod		 Bumped versions for multiple dependencies: kubescape/k8s-interface v0.0.204 → v0.0.205, kubescape/storage v0.0.247 → v0.0.252, golang.org/x/sys v0.40.0 → v0.42.0, and several go.opentelemetry.io/otel modules (various packages) updated.
Metadata Key Updates
pkg/objectcache/shared_container_data.go		 GetLabels switch cases changed to use RelatedKindMetadataKey and RelatedNameMetadataKey, deriving label values via wlid.GetKindFromWlid and wlid.GetNameFromWlid respectively.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

release

Poem

🐰 A tiny hop, dependencies climb the tree,
Related names and kinds now dance with glee,
Labels find their WLID song,
Versions hum along,
A rabbit cheers — small change, big spree! 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main changes: bumping k8s-interface dependency and updating renamed constants in the codebase.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/bump-k8s-interface-renamed-constants
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@idohuber idohuber force-pushed the fix/bump-k8s-interface-renamed-constants branch from 129e489 to f00b41a Compare March 17, 2026 17:38
@idohuber @claude
fix: bump k8s-interface to v0.0.205, storage to v0.0.252 and update r…
bcc99e3 
…enamed constants

KindMetadataKey → RelatedKindMetadataKey
NameMetadataKey → RelatedNameMetadataKey

Signed-off-by: idoh <idoh@armosec.io>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@idohuber idohuber force-pushed the fix/bump-k8s-interface-renamed-constants branch from f00b41a to bcc99e3 Compare March 17, 2026 18:19
@matthyx matthyx added release Create release and removed release Create release labels Mar 17, 2026
@matthyx matthyx moved this to Needs Reviewer in KS PRs tracking Mar 17, 2026
@matthyx
Copy link
Contributor

matthyx commented Mar 19, 2026

superseded by #754

@matthyx matthyx closed this Mar 19, 2026
@matthyx matthyx deleted the fix/bump-k8s-interface-renamed-constants branch March 19, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@shanyl9 shanyl9 shanyl9 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

KS PRs tracking
Status: Needs Reviewer

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@idohuber @matthyx @shanyl9